Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[fix] 무한 캐러셀 관련 버그 수정 #66

Merged
merged 3 commits into from
Aug 5, 2024
Merged

[fix] 무한 캐러셀 관련 버그 수정 #66

merged 3 commits into from
Aug 5, 2024

Conversation

lybell-art
Copy link
Collaborator

#️⃣ 연관 이슈

📝 작업 내용

무한 캐러셀 관련 버그를 수정했습니다.

  • 컴포넌트가 리렌더링될 시, 캐러셀이 아예 멈춰버리는 오류를 수정했습니다.
    • 실제로 60초마다 실행되는 로직 자체를 별도의 함수로 분리하고, requestAnimationFrame을 마운트하는 로직을 다른 useEffect를 써서 해결했습니다.
  • 마우스 드래그 중인 상태에서 마우스가 캐러셀 영역을 벗어났을 때, 캐러셀이 움직이는 버그를 수정했습니다.
    • 마우스 호버/호버아웃 이벤트에서 isControlled 상태를 조절하는 걸 지웠습니다.

컴포넌트가 리렌더링될 시, 코멘트 섹션이 데이터를 또 불러오는 시도를 하는 오류를 수정했습니다.

@lybell-art lybell-art added the fix 버그 및 오류 수정 label Aug 5, 2024
@lybell-art lybell-art requested a review from darkdulgi August 5, 2024 04:24
@lybell-art lybell-art self-assigned this Aug 5, 2024
darkdulgi
darkdulgi approved these changes Aug 5, 2024
View reviewed changes
Copy link
Collaborator

@darkdulgi darkdulgi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good

@darkdulgi darkdulgi merged commit d0ff732 into dev Aug 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@darkdulgi darkdulgi darkdulgi approved these changes

Assignees

@lybell-art lybell-art

Labels
fix 버그 및 오류 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lybell-art @darkdulgi